Fix FalkorDB sanitize to handle forward slash character #1118
+2
−1
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
When entity names contained forward slashes (e.g., "Cruise / En route"), RediSearch queries were failing with syntax errors because the forward slash was not being sanitized. This adds '/' to the separator map in FalkorDB's sanitize() method, treating it like other special characters by replacing it with whitespace. Fixes RediSearch query syntax errors when inserting data with forward slashes.
sgaluza
added a commit
to sgaluza/graphiti
that referenced
this pull request
Jan 14, 2026
Add forward slash (/), backslash (\), and pipe (|) to the sanitize() method to prevent RediSearch query syntax errors. RediSearch uses these characters with special meaning: - | is the OR operator - \ is the escape character - / causes query syntax errors in entity names Updated docstring with: - Reference to RediSearch escaping documentation - Note that this is for fulltext search, not exact path matching Fixes getzep#1144 Related to getzep#1118 Built with [OnSteroids](https://onsteroids.ai) Co-Authored-By: OnSteroids <[email protected]>
3 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
/to the separator map in FalkorDB'ssanitize()methodProblem
When entity names contained forward slashes (e.g., "Cruise / En route"), RediSearch queries were failing with syntax errors because the forward slash was not being sanitized. This resulted in malformed queries like:
Solution
Added
/to theseparator_mapin thesanitize()method (line 324 infalkordb_driver.py), treating it like other special characters by replacing it with whitespace.Now "Cruise / En route" is sanitized to "Cruise En route", which produces valid RediSearch queries.
Testing
Verified that:
Fixes data insertion issues when using FalkorDB backend with forward slash characters in entity names.